-
Notifications
You must be signed in to change notification settings - Fork 531
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add label to webextension add-ons on their detail page #5153
Conversation
@@ -563,14 +569,10 @@ def process_comment(self): | |||
self.version.update(**kw) | |||
self.log_action(amo.LOG.COMMENT_VERSION) | |||
|
|||
|
|||
class ReviewAddon(ReviewBase): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"""Add firefox57 tag to addons with the specified ids.""" | ||
log.info( | ||
'Adding firefox57 tag to addons %d-%d [%d].', | ||
ids[0], ids[-1], len(ids)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick, if ids
is empty this could raise an IndexError
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! lgtm, just one minor nitpick but given that it's a once-off probably not needed.
As asked in the issue, also add a tag to those add-ons to be able to filter them in search.
Fix mozilla/addons#4167